Remove admin_comment from company #3643
Open
+38
−4
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
juniwbjerde
force-pushed
the
remove-adminComment-from-companies
branch
from
e518c18
to
cfa6fed
Compare
ivarnakken
reviewed
Sep 25, 2024
ivarnakken
added
discussion
|
@ivarnakken I talked to Falk about it, and his impression after talking to Bedkom was that they didn't use it. I don't see why we would need both the admin_comment field as well as normal comments on companies. It's just a bit messy in my opinion, and there's nothing you would write in the admin comment that wouldn't fit just as well as a normal comment (in my opinion). And with normal comments you get the date it was written as well, so that you can more easily see if it's updated info or not. Espescially when we don't just delete the data in the admin_comment field completely, but rather mighrated it to a comment, I feel like it is completly fine. I don't have access to to itDAGENES admin page, how do they do it? |
juniwbjerde
force-pushed
the
remove-adminComment-from-companies
branch
from
cfa6fed
to
2948c1d
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Remove the field admin_comment from the company model. It was never really used...
I also wrote a custom data migration to make a normal comment on the company with the admin_comment (in case any of the admin_comments were actually useful and shouldn't be totally deleted
Corresponding frontend changes: webkom/lego-webapp#5001
Resolves ABA-1052